Skip to content

feat(redis): Inject creds via volume#1112

Merged
openshift-merge-bot[bot] merged 6 commits intoredhat-developer:masterfrom
olivergondza:redis-creds-via-volume
Apr 28, 2026
Merged

feat(redis): Inject creds via volume#1112
openshift-merge-bot[bot] merged 6 commits intoredhat-developer:masterfrom
olivergondza:redis-creds-via-volume

Conversation

@olivergondza
Copy link
Copy Markdown
Collaborator

@olivergondza olivergondza commented Mar 31, 2026

What type of PR is this?

Uncomment only one /kind line, and delete the rest.
For example, > /kind bug would simply become: /kind bug

/kind bug
/kind cleanup
/kind failing-test
/kind enhancement
/kind documentation
/kind code-refactoring

What does this PR do / why we need it:

Avoid passing the password over environment variables and command line arguments. Downstream of:

Have you updated the necessary documentation?

  • [no] Documentation update is required by this PR.
  • [no] Documentation has been updated.

Which issue(s) this PR fixes:

Fixes #?

Test acceptance criteria:

  • Unit Test
  • E2E Test

How to test changes / Special notes to the reviewer:

Blocked until the upstream PRs are merged.

@openshift-ci openshift-ci Bot added the kind/enhancement New feature or request label Mar 31, 2026
@openshift-ci openshift-ci Bot requested review from jannfis and trdoyle81 March 31, 2026 11:04
@olivergondza olivergondza changed the title feat(redis): Inject creds via volume (WIP) feat(redis): Inject creds via volume Apr 14, 2026
Signed-off-by: Oliver Gondža <[email protected]>
@olivergondza olivergondza force-pushed the redis-creds-via-volume branch from 91ed74d to 57ed555 Compare April 14, 2026 12:56
@olivergondza
Copy link
Copy Markdown
Collaborator Author

Test failures are relevant...

Signed-off-by: Oliver Gondža <[email protected]>
@olivergondza
Copy link
Copy Markdown
Collaborator Author

/test v4.14-kuttl-sequential

1 similar comment
@olivergondza
Copy link
Copy Markdown
Collaborator Author

/test v4.14-kuttl-sequential

@olivergondza
Copy link
Copy Markdown
Collaborator Author

/retest

anandf
anandf previously approved these changes Apr 20, 2026
Copy link
Copy Markdown
Member

@anandf anandf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@anandf
Copy link
Copy Markdown
Member

anandf commented Apr 20, 2026

/lgtm /approve

Copy link
Copy Markdown
Contributor

@nmirasch nmirasch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

…into redis-creds-via-volume

Signed-off-by: Oliver Gondža <[email protected]>
@ranakan19
Copy link
Copy Markdown

curious why code-rabbit did not pick this up

@olivergondza
Copy link
Copy Markdown
Collaborator Author

curious why code-rabbit did not pick this up

Imo we use it in argocd-operator only, not sure why

@anandf
Copy link
Copy Markdown
Member

anandf commented Apr 28, 2026

/lgtm
/approve

@openshift-ci
Copy link
Copy Markdown

openshift-ci Bot commented Apr 28, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: anandf

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-merge-bot openshift-merge-bot Bot merged commit 02400d0 into redhat-developer:master Apr 28, 2026
16 checks passed
@olivergondza olivergondza deleted the redis-creds-via-volume branch April 28, 2026 11:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants